-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
comparaminstance.spec property is not available during object creation #392
base: main
Are you sure you want to change the base?
comparaminstance.spec property is not available during object creation #392
Conversation
can you provide a traceback? (does the exception happen in your own code or somewhere within odxtools.) The point with this is that using |
The traceback is already included (screenshot) This PR just reverts the change from https://github.com/mercedes-benz/odxtools/pull/386/files#r1952936788 to unblock the situation. you can do a follow up PR with a more proper fix. |
Yes, but I'm a bit confused: for me, loading a PDX file works fine using the current |
@andlaus In Line 154 in 1e5f030
Creating the NamedItemList causes short_name getter to be called |
I understand. What I don't get is why this is a problem: the spec_ref of the comparam_refs is exclusively resolved in _resolve_odxlinks() and |
Although I tested the changes introduced in #386, after upgrading to the latest odxtools version 9.4.0, we got the issue:
I believe not encountering the issue during testing was due to a Git branching issue :( .